Skip to content

MINOR: Cleanups in ops docs #20532

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 14, 2025
Merged

MINOR: Cleanups in ops docs #20532

merged 1 commit into from
Sep 14, 2025

Conversation

mimaison
Copy link
Member

@mimaison mimaison commented Sep 12, 2025 •
edited by github-actions bot
Loading

  • Fix typo in process.role
  • Fix formatting of quorum description commands

Reviewers: Lan Ding [email protected], Ken Huang [email protected],
TengYao Chi [email protected]

@github-actions github-actions bot added docs small Small PRs labels Sep 12, 2025
DL1231
DL1231 approved these changes Sep 14, 2025
View reviewed changes
Copy link
Contributor

@DL1231 DL1231 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the improvement, LGTM.

m1a2st
m1a2st approved these changes Sep 14, 2025
View reviewed changes
Copy link
Collaborator

@m1a2st m1a2st left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

frankvicky
frankvicky approved these changes Sep 14, 2025
View reviewed changes
Copy link
Contributor

@frankvicky frankvicky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mimaison: LGTM

@frankvicky frankvicky merged commit 374bc46 into apache:trunk Sep 14, 2025
26 checks passed
jim0987795064 pushed a commit to jim0987795064/kafka that referenced this pull request Sep 25, 2025
@mimaison
MINOR: Cleanups in ops docs (apache#20532)
9b962e7 
- Fix typo in `process.role`
- Fix formatting of quorum description commands

Reviewers: Lan Ding <[email protected]>, Ken Huang <[email protected]>,
TengYao Chi <[email protected]>
chia7712
chia7712 reviewed Sep 28, 2025
View reviewed changes
docs/ops.html
<pre><code class="language-bash">
$ bin/kafka-storage.sh format -t KAFKA_CLUSTER_ID --feature kraft.version=1 -c controller.properties
</code></pre><p>
<pre><code class="language-bash">$ bin/kafka-features.sh --bootstrap-controller localhost:9093 describe</code></pre>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I cherry-pick this patch to 4.1 for #20604

chia7712 pushed a commit that referenced this pull request Sep 28, 2025
@mimaison @chia7712
MINOR: Cleanups in ops docs (#20532)
f4ce123 
- Fix typo in `process.role`
- Fix formatting of quorum description commands

Reviewers: Lan Ding <[email protected]>, Ken Huang <[email protected]>,
TengYao Chi <[email protected]>
@chia7712 chia7712 mentioned this pull request Sep 28, 2025
Merged
@mimaison mimaison deleted the ops-docs branch September 28, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chia7712 chia7712 chia7712 left review comments

@frankvicky frankvicky frankvicky approved these changes

+2 more reviewers

@DL1231 DL1231 DL1231 approved these changes

@m1a2st m1a2st m1a2st approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
docs small Small PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mimaison @chia7712 @frankvicky @DL1231 @m1a2st